Skip to content

Fix: Render KB embed text in article views - #149

Open
biast12 wants to merge 1 commit into
masterfrom
kb-browse-embed
Open

Fix: Render KB embed text in article views#149
biast12 wants to merge 1 commit into
masterfrom
kb-browse-embed

Conversation

@biast12

@biast12 biast12 commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Description

KB article rendering now includes stored embed description/fields in the message body (within a text budget) and uses safer embed access helpers to avoid nil-pointer edge cases. Article views also show a no-description placeholder only when nothing at all can be rendered, while preserving image rendering and button behavior. Additionally, the outdated /kb send TODO was removed, and ticket reopen limit replies now pass pluralized limit arguments with error styling.

Type of Change

  • Bug fix
  • New feature
  • Breaking change

Testing

Do not have any kb categories
Use the /kb browse command - you should not see the "%!(EXTRA []interface {}=[])" at the end of the message
Use the command again, this time with a kb category and a kb article
The article should not have any content other than a description and an embed

Checklist

  • My code follows the style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

KB article rendering now includes stored embed description/fields in the message body (within a text budget) and uses safer embed access helpers to avoid nil-pointer edge cases. Article views also show a no-description placeholder only when nothing at all can be rendered, while preserving image rendering and button behavior. Additionally, the outdated /kb send TODO was removed, and ticket reopen limit replies now pass pluralized limit arguments with error styling.
@biast12
biast12 requested a review from a team as a code owner September 5, 2026 18:22
@github-actions github-actions Bot added the type:bug Bug Fixes label Sep 5, 2026
@biast12 biast12 changed the title Render KB embed text in article views Fix: Render KB embed text in article views Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants